Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

Feature/mailchimp api preference api #816

Merged
merged 6 commits into from
May 25, 2016

Conversation

vikasrohit
Copy link
Contributor

@parthshah fyi
The PR includes the changes needed to adapt the frontend for the preferences based API instead of mailchimp API wrapper.

vikasrohit added 6 commits May 10, 2016 18:22
… preferences service

-- Adapted code to use the refactored api
* dev:
  AS#118090147171048, Mail chimp Integration

Conflicts:
	app/settings/email/email.controller.js
… preferences service

-- Fixed new service reference in tests
… preferences service

-- Fixed new service reference in tests
… preferences service

-- Adapted calling code for express based preference API
* dev:
  AS#125999129126033, 1 Million users hero banner on Dashboard
  AS#125999129126033, 1 Million users hero banner on Dashboard
  Updated contact us link
@vikasrohit vikasrohit merged commit f9cd962 into dev May 25, 2016
@@ -74,7 +74,7 @@ import _ from 'lodash'
case 'SUBMISSIONS':
case 'USER':
return _getRestangularV3(CONSTANTS.AUTH_API_URL)
case 'MAILCHIMP':
case 'PREFERENCES':
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this point to API_URL (api.topcoder.com) instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, you are correct. Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants